Skip to content

skip lint check#64

Merged
rajatchopra merged 1 commit into
NVIDIA:mainfrom
rajatchopra:lintfix
Apr 21, 2026
Merged

skip lint check#64
rajatchopra merged 1 commit into
NVIDIA:mainfrom
rajatchopra:lintfix

Conversation

@rajatchopra

@rajatchopra rajatchopra commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

skip lint check, rename bytes variable as it overwrites bytes package
without the # nosec: G115 marker, the linter gives this output:

golangci-lint run ./...
internal/links/pciinfo.go:37:10: G115: integer overflow conversion int8 -> byte (gosec)
		if byte(b) == '\x00' {
		       ^
internal/links/pciinfo.go:40:31: G115: integer overflow conversion int8 -> byte (gosec)
		pbytes = append(pbytes, byte(b))
		                            ^
2 issues:
* gosec: 2
make: *** [Makefile:55: golangci-lint] Error 1

Local run passes with the flags.

Signed-off-by: Rajat Chopra <rajatc@nvidia.com>
@rajatchopra rajatchopra merged commit ddc6eae into NVIDIA:main Apr 21, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants